-
Notifications
You must be signed in to change notification settings - Fork 12.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Node.js to PR CI image #69964
Add Node.js to PR CI image #69964
Conversation
This should allow the `rustdoc-js` and `rustdoc-js-std` test suites to run automatically on PRs.
(rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ rollup Seems like a minimal and easy addition. |
📌 Commit 965888a has been approved by |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors r- |
If only. These test suites apparently never supported being run from a directory other than the source code root. I've pushed a commit to hopefully fix that. |
I went to the quickest but this is really nice to make those tests being able to run with a given output folder. Were you able to test it locally without issues? If so, r=me. Again, thanks a lot for making them easier to run! |
Yeah, but more importantly the logs from the PR build: https://dev.azure.com/rust-lang/rust/_build/results?buildId=23190&view=logs&j=0230d20d-e0df-5b62-a6ec-fad0f9c3c3d5&t=a6ad31e5-07ba-5fd6-c0b6-53535f688400&l=3574 show that it successfully ran the tests. @bors r=Mark-Simulacrum,GuillaumeGomez |
📌 Commit 3f58ab6 has been approved by |
…m,GuillaumeGomez Add Node.js to PR CI image This should allow the `rustdoc-js` and `rustdoc-js-std` test suites to run automatically on PRs.
Rollup of 8 pull requests Successful merges: - #69686 (Use `pprust` to print attributes in rustdoc) - #69858 (std: on Windows, use GetSystemTimePreciseAsFileTime if it is available) - #69917 (Cleanup E0412 and E0422) - #69964 (Add Node.js to PR CI image) - #69992 (Block version-specific docs from search engines) - #69995 (Add more context to the literal overflow message) - #69998 (Add long error explanation for E0634) - #70014 (Small fixes in rustdoc book) Failed merges: r? @ghost
This should allow the
rustdoc-js
andrustdoc-js-std
test suites to run automatically on PRs.